Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

egl: set context version when egl < 1.5 #1655

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

kchibisov
Copy link
Member

Signed-off-by: p0ryae [email protected]
Fixes #1647.

--

@p0ryae does it fix your issue?

@MarijnS95
Copy link
Member

The commit signoff looks a bit weird, since you (@kchibisov) are both the author and committer.

Perhaps this should be a Reported-by:, Suggested-by:, Tested-by:?

@kchibisov
Copy link
Member Author

Perhaps this should be a Reported-by:, Suggested-by:, Tested-by:?

https://www.kernel.org/doc/html/latest/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by

@kchibisov
Copy link
Member Author

I guess C-d-b is better.

@MarijnS95
Copy link
Member

Oh yeah I forgot C-d-b, that's best!

@p0ryae
Copy link

p0ryae commented Dec 21, 2023

@p0ryae does it fix your issue?

Yes. Tested the egl-1.3-set-version branch and my tests pass.

@kchibisov kchibisov merged commit 6160cfa into rust-windowing:master Dec 21, 2023
43 checks passed
@kchibisov kchibisov deleted the egl-1.3-set-version branch December 21, 2023 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Unable to Force GLES2 Context (Still picks up the wrong GLES version)
3 participants